GtkActionBar: Just leave the center widget as NULL, initially
authorMatthias Clasen <mclasen@redhat.com>
Tue, 7 Jan 2014 21:16:52 +0000 (16:16 -0500)
committerMatthias Clasen <mclasen@redhat.com>
Tue, 7 Jan 2014 21:16:52 +0000 (16:16 -0500)
No need to construct a box that gets thrown away later.
Plus, this fixes the defaultvalue test for GtkActionBar.

gtk/gtkactionbar.c

index 527a04cbec23d43423450b4f0cf6c4bba9a7cd8d..2c810c94f4defae53586ebf214873471d0cdd103 100644 (file)
@@ -99,19 +99,6 @@ get_css_padding_and_border (GtkWidget *widget,
   border->left += tmp.left;
 }
 
-static void
-construct_center_widget (GtkActionBar *bar)
-{
-  GtkActionBarPrivate *priv = gtk_action_bar_get_instance_private (bar);
-
-  g_assert (priv->center_widget == NULL);
-
-  priv->center_widget = gtk_box_new (GTK_ORIENTATION_VERTICAL, 0);
-  gtk_widget_set_valign (priv->center_widget, GTK_ALIGN_CENTER);
-  gtk_widget_show (priv->center_widget);
-  gtk_widget_set_parent (priv->center_widget, GTK_WIDGET (bar));
-}
-
 static void
 gtk_action_bar_init (GtkActionBar *bar)
 {
@@ -127,8 +114,6 @@ gtk_action_bar_init (GtkActionBar *bar)
   priv->children = NULL;
   priv->spacing = DEFAULT_SPACING;
 
-  construct_center_widget (bar);
-
   context = gtk_widget_get_style_context (GTK_WIDGET (bar));
   gtk_style_context_add_class (context, "action-bar");
   gtk_style_context_add_class (context, GTK_STYLE_CLASS_HORIZONTAL);
@@ -618,10 +603,6 @@ gtk_action_bar_set_center_widget (GtkActionBar *bar,
       gtk_widget_set_valign (priv->center_widget, GTK_ALIGN_CENTER);
       gtk_widget_show (center_widget);
     }
-  else
-    {
-      construct_center_widget (bar);
-    }
 
   gtk_widget_queue_resize (GTK_WIDGET (bar));